Added build-nightly_13.yml#3444
Conversation
Upgraded used GHA to the latest versions
📝 WalkthroughWalkthroughGitHub Actions workflow files updated across three configurations: version upgrades applied to checkout (v4→v6) and upload-artifact (v4→v7) actions in existing nightly and coverage workflows, plus a new nightly workflow added for version 13.0 with branch validation. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan for PR comments
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.github/workflows/build-nightly_13.yml (1)
36-36: Deprecated bash arithmetic syntax in commented code.The
$[...]syntax is deprecated in modern bash. When this code is uncommented, consider using$((...))instead for arithmetic expansion.-# timespan=$[now - commit] +# timespan=$((now - commit))Note: The same deprecated syntax exists in
build-nightly_12.3.ymlat line 35 and could be fixed there as well.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/workflows/build-nightly_13.yml at line 36, The commented arithmetic assignment timespan=$[now - commit] uses deprecated $[...] syntax; update the comment to use POSIX-friendly arithmetic expansion by replacing the deprecated form with $((now - commit)) wherever it appears (including the similar occurrence in build-nightly_12.3.yml) so that when uncommented it will work in modern bash.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.github/workflows/build-nightly_13.yml:
- Line 36: The commented arithmetic assignment timespan=$[now - commit] uses
deprecated $[...] syntax; update the comment to use POSIX-friendly arithmetic
expansion by replacing the deprecated form with $((now - commit)) wherever it
appears (including the similar occurrence in build-nightly_12.3.yml) so that
when uncommented it will work in modern bash.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: f0eb4b11-7d9d-46fb-aa61-626ac9e377ae
📒 Files selected for processing (3)
.github/workflows/build-nightly_12.3.yml.github/workflows/build-nightly_13.yml.github/workflows/coverage.yml
msevestre
left a comment
There was a problem hiding this comment.
assuming the code in 13 is the same as in 12.3 except version
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.github/workflows/build-nightly_v13.yml (1)
11-20: Consider using an environment variable for safer shell interpolation.While the current implementation works correctly, directly interpolating
${{ github.ref_name }}into the shell script could be fragile if branch names ever contain special characters. Using an environment variable is a more robust pattern.♻️ Suggested improvement
jobs: check-branch: runs-on: ubuntu-latest steps: - name: Validate source branch + env: + BRANCH_NAME: ${{ github.ref_name }} run: | - if [ "${{ github.ref_name }}" != "V13" ]; then + if [ "$BRANCH_NAME" != "V13" ]; then echo "::error::Please call the workflow from the V13 branch" exit 1 fi🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/workflows/build-nightly_v13.yml around lines 11 - 20, The step "Validate source branch" in job check-branch currently interpolates ${{ github.ref_name }} directly into the shell; instead set an environment variable (e.g., REF_NAME) from ${{ github.ref_name }} at the job or step level and reference that in the run script using a quoted shell variable ("$REF_NAME") to avoid word-splitting or special-character issues; update the run block in the "Validate source branch" step to use the env var and ensure comparisons use the quoted variable.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.github/workflows/build-nightly_v13.yml:
- Around line 11-20: The step "Validate source branch" in job check-branch
currently interpolates ${{ github.ref_name }} directly into the shell; instead
set an environment variable (e.g., REF_NAME) from ${{ github.ref_name }} at the
job or step level and reference that in the run script using a quoted shell
variable ("$REF_NAME") to avoid word-splitting or special-character issues;
update the run block in the "Validate source branch" step to use the env var and
ensure comparisons use the quoted variable.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 28cfc06d-a931-4243-a51f-ac7b78ecc713
📒 Files selected for processing (1)
.github/workflows/build-nightly_v13.yml
Actually we don't need to do anything in build-nightly_v13.yml on the develop branch (except for the branch check). |
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change
Please mark relevant options with an
xin the brackets.How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Reviewer checklist
Mark everything that needs to be checked before merging the PR.
This change requires a documentation updateabove is selectedScreenshots (if appropriate):
Questions (if appropriate):
Summary by CodeRabbit